Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

billy/ 521 Consistent Search #758

Merged
merged 7 commits into from
May 30, 2018
Merged

billy/ 521 Consistent Search #758

merged 7 commits into from
May 30, 2018

Conversation

okwme
Copy link
Contributor

@okwme okwme commented May 29, 2018

Closes #521

Description:
made search consistent as per @jolesbi recommendations on #521

❤️ Thank you!

@okwme okwme requested a review from jbibla May 29, 2018 10:39
@okwme okwme changed the title enough test coverage? billy/ 521 Consistent Search May 29, 2018
@codecov
Copy link

codecov bot commented May 29, 2018

Codecov Report

Merging #758 into develop will decrease coverage by 0.11%.
The diff coverage is 95.45%.

@@             Coverage Diff             @@
##           develop     #758      +/-   ##
===========================================
- Coverage    87.87%   87.75%   -0.12%     
===========================================
  Files           94       94              
  Lines         1583     1601      +18     
  Branches        73       88      +15     
===========================================
+ Hits          1391     1405      +14     
- Misses         181      182       +1     
- Partials        11       14       +3
Impacted Files Coverage Δ
...p/src/renderer/components/common/NiModalSearch.vue 83.33% <ø> (+11.11%) ⬆️
app/src/renderer/vuex/modules/wallet.js 88.46% <100%> (+0.22%) ⬆️
app/src/renderer/vuex/modules/blockchain.js 97.61% <100%> (+0.05%) ⬆️
app/src/renderer/components/wallet/PageWallet.vue 94.87% <100%> (+0.42%) ⬆️
...pp/src/renderer/components/staking/PageStaking.vue 97.14% <100%> (-2.86%) ⬇️
app/src/renderer/components/monitor/PageBlocks.vue 95.45% <100%> (-4.55%) ⬇️
app/src/renderer/vuex/modules/delegates.js 100% <100%> (ø) ⬆️
...rc/renderer/components/wallet/PageTransactions.vue 96.15% <100%> (-3.85%) ⬇️
...p/src/renderer/components/govern/PageProposals.vue 38.7% <66.66%> (-4.15%) ⬇️
... and 5 more

@jbibla
Copy link
Collaborator

jbibla commented May 29, 2018

hey @okwme we have a "codecov can't go down" policy so i'll add some tests!

jbibla
jbibla previously approved these changes May 29, 2018
@okwme
Copy link
Contributor Author

okwme commented May 30, 2018

@jolesbi bless u for the bump!

yours didn't pull us out of the red tho did it? was also surprised mine was in red after the first fail and the subsequent beefing that already took place...

@jbibla
Copy link
Collaborator

jbibla commented May 30, 2018

i wanted to get it in! but no, i couldn't get it through... seems very strange to me.

@faboweb
Copy link
Collaborator

faboweb commented May 30, 2018

Could you improve the coverage for PageBlocks, PageTransactions, PageStaking (probably copy pasting the test for somethingToSearch)?

@okwme
Copy link
Contributor Author

okwme commented May 30, 2018

the only thing not covered in PageBlocks, PageTransactions or PageStaking is Mousetrap. We shouldn't be testing that tho, right?

PageBlocks:
screenshot 2018-05-30 15 25 38
PageTransactions :
screenshot 2018-05-30 15 26 42
PageStaking:
screenshot 2018-05-30 15 27 16

@jbibla
Copy link
Collaborator

jbibla commented May 30, 2018

Could you improve the coverage for PageBlocks, PageTransactions, PageStaking (probably copy pasting the test for somethingToSearch)?

i did that here 1ff25d1

@faboweb
Copy link
Collaborator

faboweb commented May 30, 2018

Very good PR. :)

@faboweb faboweb merged commit dad7e0c into develop May 30, 2018
@faboweb faboweb deleted the billy/521-consistent-search branch May 30, 2018 13:42
faboweb pushed a commit that referenced this pull request Jun 2, 2020
Bumps [eslint](https://github.com/eslint/eslint) from 6.7.0 to 7.0.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](eslint/eslint@v6.7.0...v7.0.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants